Skip to content

[ALICE3] Refactor TRK Hit class to rely on ITSMFT#15173

Merged
mconcas merged 3 commits intodevfrom
njacazio-patch-2
Mar 18, 2026
Merged

[ALICE3] Refactor TRK Hit class to rely on ITSMFT#15173
mconcas merged 3 commits intodevfrom
njacazio-patch-2

Conversation

@njacazio
Copy link
Collaborator

No description provided.

@github-actions
Copy link
Contributor

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

Removed unnecessary includes and simplified the Print method.
Removed the declaration of the Hit class from TimeFrame.h.
@njacazio njacazio marked this pull request as ready for review March 17, 2026 05:44
@njacazio njacazio requested a review from mconcas as a code owner March 17, 2026 05:44
@njacazio njacazio enabled auto-merge (squash) March 18, 2026 05:34
@njacazio njacazio disabled auto-merge March 18, 2026 05:44
@njacazio njacazio enabled auto-merge (squash) March 18, 2026 05:44
@mconcas
Copy link
Collaborator

mconcas commented Mar 18, 2026

Errors for aarch64 seem genuine but not related to something that this PR touches. Merging this anyways, @njacazio pleas have a look.

@mconcas mconcas disabled auto-merge March 18, 2026 08:07
@mconcas mconcas merged commit 8e4cfe1 into dev Mar 18, 2026
12 of 13 checks passed
@mconcas mconcas deleted the njacazio-patch-2 branch March 18, 2026 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants